[FEATURE] added composer for managing external libraries #27
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
instead of processing the whole music collection within on mysql-fetch-loop smaller batchsizes gets processed. now there is NO(!!!) memory growth within importer-phase fileInfo() at all. Unfortunately the memory-usage of the previous importer-phase (fileStructure) is still a problematic starting point for fileInfo() further some massive code-cleanup regarding tag-processing had been done. TODO: make use of the new tag-processing where needed to remove duplicate code-mess TODO: refactoring fileStructure() in terms of memory consumption
|
Hi Artur, |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
added composer for managing external libraries.
This pull-request is related to #26, #21
on your development instances you can run
./composer.phar updateto fetch the configured versions of external libraries. To keep O!MPD's repo size as small as possible i have added an additional deploy-script which copies only required files fromvendorintovendor-distbased onscripts/deploy-vendor.txt. Stuff like demos, unminified version, git-repos, etc won't be included in O!MPD repositoryso after adding/updating external libraries you have to run
./scripts/deploy-vendor.sh